Skip to content

[201911][Monit] Monitor critical processes in radv and dhcp_relay containers.#7340

Merged
yozhao101 merged 3 commits intosonic-net:201911from
yozhao101:monitor_radv_dhcp
Apr 16, 2021
Merged

[201911][Monit] Monitor critical processes in radv and dhcp_relay containers.#7340
yozhao101 merged 3 commits intosonic-net:201911from
yozhao101:monitor_radv_dhcp

Conversation

@yozhao101
Copy link
Contributor

Signed-off-by: Yong Zhao [email protected]

Why I did it

This PR aims to monitor critical processes in router advertiser and dhcp_relay containers by Monit.

How I did it

Router advertiser container only ran on T0 device and the T0 device should have at least one VLAN interface
which was configured an IPv6 address. At the same time, router advertiser container will not run on devices of which
the deployment type is 8.

As such, I created a service which will dynamically generate Monit configuration file of router advertiser from a
template.

Similarly Monit configuration file of dhcp_relay was also generated from a template since the number of dhcrelay process in dhcp_relay container is depended on number of VLANs.

How to verify it

I verified this implementation on a DuT.

Which release branch to backport (provide reason below if selected)

  • 201811
  • [x ] 201911
  • 202006
  • 202012

Description for the changelog

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Yong Zhao <[email protected]>
@yozhao101 yozhao101 merged commit 528543b into sonic-net:201911 Apr 16, 2021
@rlhui
Copy link
Contributor

rlhui commented Apr 16, 2021

could we please paste what's the corresponding sonic-mgmt test case for this?

@yozhao101
Copy link
Contributor Author

could we please paste what's the corresponding sonic-mgmt test case for this?

I am working on the PR of pytest script and will paste the link of PR later.

@yozhao101 yozhao101 deleted the monitor_radv_dhcp branch April 16, 2021 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants